Fix/issue170 by CaseAsLimbo · Pull Request #171 · profcomff/rating-api

CaseAsLimbo · 2026-05-19T19:17:32Z

Изменения

Проблема падения 57 тестов с ERROR(Исправить падающие тесты #170)
- Был исправлен баг из-за которого после падения не анониманого комментария с ERROR падало 57 тестов
- Была исправлена причина падения теста test_create_comment с тестовым набором включающим не анонимные комментарии
- Добавлена проверка полей добавляемых комментариев на соответствие userdata
Написание тестов для ручки import_comments(Написать тесты для ручки post comment/import)
- Добавлены тесты для ручки post comment/import
Был реализован мок aiohttp запросов(для выдачи ачивки в test_comment_create) через библиотеку aioresponses

Детали реализации

Баг из-за которого падало 57 тестов с ERROR связан с не очевидным поведением sqlalchemy, где на основе того, были ли подготовлены на вставку (INSERT) в текущей не завершенной транзакции одновременно записи в таблицах LecturerUserComment и Comment. Если тест падал с TypeError и успевала добавиться только одна из этих таблиц, алхимия в tear-down фикстуры lecturers в итоговый SQL запрос не добавляла строку на удаление записи LecturerUserComment - сразу после этого происходило удаление лектора из таблицы Lecturer, что приводило к конфликту ведь не удаленный LecturerUserComment по-прежнему ссылался на него.
Решение: (update) В фикстуре lecturers добавлен dbsession.flush() после добавления в транзакцию каждого элемента на удаление.
Тест test_create_comment с не анонимаными комментариями падал, из-за того, что для user-а передавалась не корректная userdata. Были добавлены мок фикстуры authlib_user, authlib_mock и user_mock для подмены данных user-а возвращаемых UnionAuth

Check-List

Вы проверили свой код перед отправкой запроса?
Вы написали тесты к реализованным функциям?
Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

…дготовленных SQL-запросов и override_dbsession_in_route, мок для переопределения сессии ручки на тестовую сессию

…стройкой cascade, что дать алхимии понять в каком порядке их удалять, вне зависимости от связанных других таблиц

…ра uv

…turerUserComments

…ACK в фикстуру logging_sql_req_before_execute

github-actions · 2026-05-19T19:19:57Z

Coverage Report

File	Stmts	Miss	Cover	Missing
rating_api
__main__.py	4	4	0%	1–7
exceptions.py	43	11	74%	24, 35–37, 48–50, 58, 70, 78, 86
rating_api/models
base.py	64	32	50%	35–38, 51–59, 63–84, 89–94
db.py	162	30	81%	115, 117, 119, 121, 132, 167, 171, 175–177, 181–183, 187–189, 199, 213, 222–224, 229, 250, 254, 258, 272–284
rating_api/routes
base.py	16	1	94%	41
comment.py	151	120	21%	59–155, 167–176, 190–197, 246–304, 327–332, 350–370, 393–404, 429–453
exc_handlers.py	32	7	78%	29, 36, 43, 50, 64, 71, 78
lecturer.py	109	46	58%	41–48, 61–93, 228–244, 258–269
rating_api/schemas
base.py	12	4	67%	6–9
models.py	155	3	98%	189, 191, 201
rating_api/utils
mark.py	8	8	0%	1–20
TOTAL	790	266	66%

Summary

Tests	Skipped	Failures	Errors	Time
94	0 💤	52 ❌	0 🔥	16.856s ⏱️

petrCher · 2026-06-24T12:59:09Z

у меня еще локально при тестировании такие ошибки, у тебя есть? проблема в том что есть незамоканный вызов по https чего не должно быть при тестировании. мы должны без http запросов работать, для этого и созданы моки В общем посмотри, подумай что не так и как лучше исправить

… юзера; оставлена модульная структура мока юзера, чтобы не хардкодить данные userdata в тестах

…a на соответствие замоканым

github-actions · 2026-06-27T14:01:04Z

💩 Code linting failed, use black and isort to fix it.

github-actions · 2026-06-27T15:26:49Z

💩 Code linting failed, use black and isort to fix it.

github-actions · 2026-06-27T15:27:39Z

💩 Code linting failed, use black and isort to fix it.

…нта в фикстуре lecturers

petrCher

у тебя мок всегда возвращает наличие ачивки, игнорируя параметр achievement_id
надо корректно проверять обе ветки логики: когда ачивки нет — POST запрос отправляется, когда есть — не отправляется

petrCher · 2026-06-29T12:46:07Z

+    check_post_response = aiohttp_mp.post(
+        settings.API_URL
+        + f"achievement/achievement/{settings.FIRST_COMMENT_ACHIEVEMENT_ID}/reciever/{authlib_user.get('id'):}",
+        status=200,


здесь лучше писать status=status.HTTP_200_OK

petrCher · 2026-06-29T12:52:06Z

        comment = Comment.query(session=dbsession).filter(Comment.uuid == post_response.json()["uuid"]).one_or_none()
        assert comment is not None

+        # проверка корректной записи user_id и fullname при анонимных и не анонимных комментариях


в наших тестах если не передано is_anonymous то он по умолчанию true
то есть даже при отсутствии его в теле запроса надо проверять, что у нас реально не сохраняется имя пользователя и остается анонимным коммент

petrCher · 2026-06-29T12:53:01Z

        assert user_comment is not None

+        # Проверка логики выдачи ачивок
+        if check_get_response.called:


а зачем это проверка? у нас же всегда выполняется check_get_response.called судя по коду
то есть можно просто убрать, здесь всегда выполнено

petrCher · 2026-06-29T12:56:17Z

+
    if response_status == status.HTTP_200_OK:
        comment = Comment.query(session=dbsession).filter(Comment.uuid == post_response.json()["uuid"]).one_or_none()
        assert comment is not None


здесь еще я бы добавил проверку на правильный статус
https://github.com/profcomff/rating-api/blob/main/rating_api/routes/comment.py#L130

petrCher · 2026-06-29T13:09:15Z

и еще, переименуй пр, пожалуйста, в более содержательное название

petrCher · 2026-06-29T13:19:42Z

https://github.com/profcomff/rating-api/actions/runs/28372071943/job/84052082838?pr=171
(см print report)
Там все еще достаточное количество ошибок, в основном 422 из-за неправильного формата данных. постарался достаточно глубоко поресерчить, очень вероятно причина в datetime в новой версии pydantic
Понял, почему у меня (и возможно у тебя) локально этого не возникает.
Надо сравнивать версии библиотек которые в CI здесь используются и у себя. Я, например, пользуюсь pytest без виртуального окружения, поэтому те библиотеки что у меня давно устарели, а в прошлых версиях регулярно были warnings про datetime. Но, возможно, помимо datetime есть еще что-то.
Так что, пожалуйста, проверь так ли и с теми же библиотеками запускается у тебя pytest и постарайся разобраться где и как поменять формат данных для корректной валидации под новые версии библиотек.

petrCher · 2026-06-29T13:22:08Z

для того, чтобы понять 422 из-за чего возникает, просто сделай отладку виде проверки if responce==422 то сделать принт

petrCher · 2026-06-29T13:27:48Z

эти две проблемы в идеале бы тоже поправить, но делать это лучше будет после исправления основной проблемы

petrCher · 2026-06-29T13:28:18Z

так что сейчас надо исправить все что выше написал + пару моментов по комментам в коде и будет мердж

CaseAsLimbo added 8 commits May 18, 2026 21:20

В .gitignore добавлено исключение файла uv.lock для пакетного менедже…

47b118c

…ра uv

Добавлен мок пользователя с userdata

54ccba8

Изменил названия атрибута lecturer_comments на lecturer в таблице Lec…

944ae37

…turerUserComments

Добавлена проверка корректности переданных в userdata параметров

b00ad8b

Были добавлены дополнительные слушатели событий BEGIN, COMMIT и ROLLB…

e5d6cf1

…ACK в фикстуру logging_sql_req_before_execute

Были применены black и isort T_T

810eeb4

CaseAsLimbo requested a review from petrCher May 19, 2026 19:17

CaseAsLimbo linked an issue May 19, 2026 that may be closed by this pull request

Исправить падающие тесты #170

Open

CaseAsLimbo added 2 commits May 19, 2026 23:12

Удалил личную информацию

a118c23

были удалены фикстры для логирования и переопределения dbsession

0342965

petrCher requested changes Jun 24, 2026

View reviewed changes

Comment thread rating_api/models/db.py Outdated

CaseAsLimbo added 3 commits June 27, 2026 16:13

Убрано каскадное удаление таблиц

81d86c5

Добавленая проверка user_id и fullname; убрана проверка полей userdat…

ad15093

…a на соответствие замоканым

CaseAsLimbo added 2 commits June 27, 2026 17:02

Применены black и isort T_T

39275f0

удален uv.lock из gitignore

fda3bc8

CaseAsLimbo linked an issue Jun 27, 2026 that may be closed by this pull request

Написать тесты для ручки post comment/import #72

Open

petrCher added 2 commits June 27, 2026 17:42

fix status

e23ca7c

fixes

d1adcf8

fix

887df60

petrCher added 2 commits June 27, 2026 18:28

fix

2e8f230

fix

6f19b2f

CaseAsLimbo added 4 commits June 27, 2026 19:12

Добавлен dbsession.flush() после добавления на удаление каждого элеме…

2260655

…нта в фикстуре lecturers

добавлен пакет aioresponses для перехвата aiohttp запросов

42e40df

Добавлена фикстура aiohttp_mp реализующая перехват всех aiohttp запросов

9eb5806

Добавлены тесты на логику выдачи ачивок

1a635eb

CaseAsLimbo requested a review from petrCher June 28, 2026 18:46

petrCher added 5 commits June 29, 2026 14:37

checks

3f8e3b8

revert

4c67b08

Update checks.yml

22982f3

Update checks.yml

67d43ae

fix

7ec71c2

petrCher requested changes Jun 29, 2026

View reviewed changes

Uh oh!

Conversation

CaseAsLimbo commented May 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Изменения

Детали реализации

Check-List

Uh oh!

github-actions Bot commented May 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

petrCher commented Jun 24, 2026

Uh oh!

Uh oh!

github-actions Bot commented Jun 27, 2026

Uh oh!

github-actions Bot commented Jun 27, 2026

Uh oh!

github-actions Bot commented Jun 27, 2026

Uh oh!

petrCher left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

petrCher Jun 29, 2026

Choose a reason for hiding this comment

Uh oh!

petrCher Jun 29, 2026

Choose a reason for hiding this comment

Uh oh!

petrCher Jun 29, 2026

Choose a reason for hiding this comment

Uh oh!

petrCher Jun 29, 2026

Choose a reason for hiding this comment

Uh oh!

petrCher commented Jun 29, 2026

Uh oh!

petrCher commented Jun 29, 2026

Uh oh!

petrCher commented Jun 29, 2026

Uh oh!

petrCher commented Jun 29, 2026

Uh oh!

petrCher commented Jun 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

CaseAsLimbo commented May 19, 2026 •

edited

Loading

github-actions Bot commented May 19, 2026 •

edited

Loading

petrCher left a comment •

edited

Loading